Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Process remove with form data asynchronously #514

Merged
merged 1 commit into from
Jan 10, 2015

Conversation

Martii
Copy link
Member

@Martii Martii commented Jan 10, 2015

  • Add a more useful status code/message for missing header instead of default
  • Some STYLEGUIDE.md conformance with line breaks, white-space and informative comments

Applies to #261 (Temporary #262 section block bypass approved by sizzle)

* Add a more useful status code/message for missing header instead of default
* Some STYLEGUIDE.md conformance with line breaks, white-space and informative comments

Applies to OpenUserJS#261 *(Temporary OpenUserJS#262 section block bypass approved by sizzle)*
Martii added a commit that referenced this pull request Jan 10, 2015
Process remove with form data asynchronously

Auto-merge... thanks to @jerone for this catch.
@Martii Martii merged commit 2075f43 into OpenUserJS:master Jan 10, 2015
@Martii Martii deleted the Issue-261asyncFix branch January 10, 2015 17:36
@Martii Martii added bug You've guessed it... this means a bug is reported. CODE Some other Code related issue and it should clearly describe what it is affecting in a comment. labels Jan 10, 2015
Martii pushed a commit to Martii/OpenUserJS.org that referenced this pull request Oct 26, 2015
* Change route to use regex and a POST... removed TODO note on it... **NOTE** `nav-pills` UI issue with form buttons... so making `Flag` and `Unflag` in their own div tag as intended
* Establish a controller function to flag/unflag
* Removed now dead code... aOptions and exports
* Cleaned up remove controller function code a little bit to be more symmetrical... post OpenUserJS#510, OpenUserJS#513, OpenUserJS#514 with parent of OpenUserJS#261 e.g. e0929ca...e1e3f7e excluding OpenUserJS#511
* Some STYLEGUIDE.md conformance
* Don't `return` some assumed `undefined`s in `aNext()`s and `statusCodePage()`

Applies to OpenUserJS#775
@github-actions github-actions bot locked as resolved and limited conversation to collaborators May 21, 2021
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
bug You've guessed it... this means a bug is reported. CODE Some other Code related issue and it should clearly describe what it is affecting in a comment.
Development

Successfully merging this pull request may close these issues.

1 participant